Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update plugin to sbt 1.0.2 #9

Merged
merged 15 commits into from
Feb 12, 2018
Merged

Update plugin to sbt 1.0.2 #9

merged 15 commits into from
Feb 12, 2018

Conversation

rhpvorderman
Copy link
Contributor

@rhpvorderman rhpvorderman commented Jan 11, 2018

1.0.2 because 1.1 gave some additional compatibility issues. Also 1.1 is very new (< 2 weeks), so better stick with the more stable 1.0

This pull request is almost complete. Things required before pulling:

  • Laika site support for sbt 1.0 sbt/sbt-site#125 must be fixed.
  • A new version of sbt-site must be released. A polite request will probably be successfull, since the last release was some time ago and the above pull request adds some nice functionality.
  • The version of sbt-site in build.sbt needs to be updated. Now it is set to a snapshot version.

@jonas
Copy link

jonas commented Feb 12, 2018

sbt-site 1.3.2 has been released.
Thanks for helping out, @rhpvorderman!

@@ -1,6 +1,6 @@
sys.props.get("plugin.version") match {
case Some(x) => addSbtPlugin("com.github.biopet" % "sbt-biopet" % x)
case _ => addSbtPlugin("com.github.biopet" % "sbt-biopet" % "0.2-SNAPSHOT")
case _ => addSbtPlugin("com.github.biopet" % "sbt-biopet" % "0.4-SNAPSHOT")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this be done with a variable? This seems likely to break in next versions.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@ffinfo ffinfo merged commit 9c74096 into develop Feb 12, 2018
@ffinfo ffinfo deleted the BIOPET-927 branch February 12, 2018 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants